Skip to content

Moved assignment endpoints to /api/assignments#35

Merged
Ke-vin-S merged 1 commit into
mainfrom
fix/assignment-path
May 9, 2025
Merged

Moved assignment endpoints to /api/assignments#35
Ke-vin-S merged 1 commit into
mainfrom
fix/assignment-path

Conversation

@Ke-vin-S

@Ke-vin-S Ke-vin-S commented May 9, 2025

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings May 9, 2025 05:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the assignment API endpoints by moving them to be under the /api/assignments/ route.

  • Updated the logistics_core/urls.py file to change the include path from /api/assignment/ to /api/assignments/.
  • Modified the router registration in assignment/urls.py to remove the redundant "assignments" prefix.
  • Adjusted all test cases in the assignment test files to reflect the new endpoint structure.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
logistics_core/urls.py Changed the assignment endpoint path to /api/assignments/
assignment/urls.py Updated router registration to remove the duplicate URL segment
assignment/tests/test_assignment_complete.py Updated test URLs to use /api/assignments/ for assignment actions
assignment/tests/test_assignment_api.py Adjusted test URLs to match the new endpoint structure

@Ke-vin-S Ke-vin-S merged commit 13f7653 into main May 9, 2025
1 check passed
@Ke-vin-S Ke-vin-S deleted the fix/assignment-path branch May 9, 2025 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants